Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make mainloop schedule type available as GemmKernel::Schedule #1638

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

manishucsd
Copy link
Contributor

This PR makes mainloop schedule uniformly available for mainloop variants. Our usage of this type involves additionally having it as a part of GemmDescription object for our kernels and decide mainloop variant at the dispatch time.

This was already there in sm90_gemm_array_tma_warpspecialized_cooperative

cc: @IonThruster , @thakkarV , @hwu36 , @mnicely

@manishucsd manishucsd changed the title Create a type for mainloop schedule uniformly available in as GemmKernel::Schedule in all variants Make mainloop schedule type available in as GemmKernel::Schedule Jul 16, 2024
@manishucsd manishucsd changed the title Make mainloop schedule type available in as GemmKernel::Schedule Make mainloop schedule type available as GemmKernel::Schedule Jul 16, 2024
@@ -34,7 +34,7 @@
#include <cutlass/library/types.h>
#include <cutlass/blas3_types.h>
#include <cutlass/gemm_coord.h>

#include <stdexcept>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Include for std::runtime_error used in this file. This will allow us to include this file in other projects without have #include <stdexcept> above its include.

Copy link

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant